Skip to content
This repository has been archived by the owner on Feb 10, 2022. It is now read-only.

Validation for optional value. #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pmercadocl
Copy link

@pmercadocl pmercadocl commented Feb 13, 2020

As described in documentation https://developers.mercadolibre.com.ar/es_ar/autenticacion-y-autorizacion#Refresh-token , application must have selected option "offline_access" to return a value, so this commit check if variable exist and is not empty. Meli library fail if application in Mercado Libre backend does not have set "offline_access" option.

This should fix #69

As described in documentation https://developers.mercadolibre.com.ar/es_ar/autenticacion-y-autorizacion#Refresh-token , application must have selected option "offline_access" to return a value, so this commit check if variable exist and is not empty. Meli library fail if application in Mercado Libre backend does not have set "offline_access" option.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refresh_token
1 participant